Skip to content

fix: dclogin links for IP addresses need square brackets#20

Open
missytake wants to merge 2 commits intomainfrom
dclogin-no-dns
Open

fix: dclogin links for IP addresses need square brackets#20
missytake wants to merge 2 commits intomainfrom
dclogin-no-dns

Conversation

@missytake
Copy link
Copy Markdown

Related to chatmail/relay#919 (review)

Not sure if the second commit should be in this PR, or why it's 12 by default; I think it just wasn't used until now? Anyway, as I see no way to set the value from chatmail.ini, let's just use the default value 9.

Copy link
Copy Markdown

@j4n j4n left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me, works for pinging my new v4 relay as well :)

Comment thread cmping.py Outdated
Co-authored-by: j4n <j4n@systemli.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants